http://pastebin.com/XpEBHB0K
בהערות למעלה ישנם הוראות התקנה.
אשמח לתגובות.

4 תשובות

avatar ענה intval ב 12 לפברואר 2012 #

כל הכבוד. יש כמה הערות קטנות:

1. קודם כל יש סטנדרטים כלתיבת הערות. נקראים phpdoc
2. במקום file_exists תבדוק is_dir ותזרוק שגיאה, לא die
3. למה שהקונסטקרטוק לא יקבל גם מערך של options ותחבר אותו עם המערך של ברירת המחדל
4. לא ברור ה submit_form בתור action של טופס אני מצפה שזו תהיה מטודה פשוטה שבסה"כ מחזירה מחרוזת ולא עושה מליון פעולות אחרות שאיכשהו מכל זה נוצר action לטופס.
5. המטודה שמדפיסה את הטופס יכולה להיות סטטית?

avatar ענה hexdec ב 12 לפברואר 2012 #

תודה רבה אלכס, אני ממש אוהב את התגובות הבונות שלך :)!

1. אפשר איזשהו קישור? לעולם לא שמעתי על זה.
2. אוקיי, ובקשר ללזרוק שגיאה.. אתה מתכוון exception & throw? אני אצטרך ללמוד על זה, מכוון שלעולם לא חשבתי שזה שימושי... למה דווקא את זה ולא die?
3. כי עדיף שזה יעשה לבד, ואם המשתמש ירצה, שישנה אותם. לא כולם מבינים בדברים האלה כל כך (למרות שצריך ידע ממש בסיסי, אבל בסדר...)
4. לא ממש הבנתי אותך. וזה פשוט מחזיר מחרוזת פשוטה של טופס. ה "פעולה" (יותר נכון מתודה) שיש שם זה קריאה למתודה אחרת שמוודאת את הטופס.
5. למה לעשות את זה?

avatar ענה intval ב 13 לפברואר 2012 #

1. gooo..e
2. die קוטע לך את הסקריפט. שגיאות לא.
3. באופן כללי אף אחד חוץ ממך לא ישתמש במחלקה הזאת, אז אתה יכול לעשות מה שאתה רוצה, למרות שנראה לי הגיוני להעביר פרמטרים לקונסטרקור בשורה אחת מאשר להפעיל קונסטרקטור ואחריו את ה setoption
4.

action="'.$this->submit_form().'"
והמטודה הזו מחזירה true או false ובכלל לא מחרוזת.
5. כי היא לא עושה שום דבר שבאמת קשור למאפייני המחלקה ולא משנה איזה מופע זה - התוצאה תמיד תהיה אותו דבר.

avatar ענה hexdec ב 13 לפברואר 2012 #

1. תודה.
2. לא ממש הצלחתי להשתמש בזריקת שגיאות וזה..
3. אחשוב על זה.
4. לא הבנתי אותך, אז מה כן לעשות? action לדף שונה? לדוגמה upload.php?
5. אם לקלאס יש כבר Instance אז למה רק את ה submit_form להפוך לסטטי? במה זה יעזור? יוריד את כמות המשאבים? לא נראה לי...